Skip to content

Conversation

@deansheather
Copy link
Member

Reworks the TestHttpServer threading to:

  • Use an async method instead (so we can wait with a cancellation token)
  • Use a cancellation token while awaiting the listener to return a request
    • GetContext() doesn't take a CTS directly, so we call Task.WaitAsync(CancellationToken ct) instead
  • Adds some exception handling and discarding where necessary to ignore expected errors during shutdown

Closes coder/internal#603

@deansheather deansheather enabled auto-merge (squash) October 28, 2025 06:08
@deansheather deansheather merged commit e4408b8 into main Oct 28, 2025
3 checks passed
@deansheather deansheather deleted the dean/flake-603 branch October 28, 2025 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

flake: ValidationFailureExistingFile

2 participants